Skip to content

Conversation

@ckenst
Copy link
Member

@ckenst ckenst commented Sep 11, 2025

Trying to use GitHub Actions rather than CircleCI

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the CI/CD pipeline from CircleCI to GitHub Actions. The CircleCI configuration is temporarily renamed (from config.yml to temp-rename-config.yml) to disable it while testing the new GitHub Actions workflow.

Key Changes:

  • New GitHub Actions workflow created to build and test the Jekyll site
  • CircleCI configuration temporarily disabled via file rename
  • Both configurations maintain the same build steps: dependency installation, Jekyll build, and HTML validation with htmlproofer

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/ci.yml New GitHub Actions workflow that builds the Jekyll site and runs htmlproofer tests on push to master and pull requests
.circleci/temp-rename-config.yml CircleCI configuration renamed to temporarily disable it during the migration testing phase

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +23 to +25
gem install bundler
bundle install

Copy link

Copilot AI Jan 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bundler installation is redundant. The ruby/setup-ruby action with bundler: 2.4.17 already installs the specified bundler version. Remove the 'gem install bundler' line as it's unnecessary and could potentially install a different bundler version than specified.

Suggested change
gem install bundler
bundle install
bundle install

Copilot uses AI. Check for mistakes.
Comment on lines +20 to +24

- name: Install dependencies
run: |
gem install bundler
bundle install
Copy link

Copilot AI Jan 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding 'bundler-cache: true' to the ruby/setup-ruby action. This will automatically cache gems based on the Gemfile.lock, significantly speeding up subsequent workflow runs. The ruby/setup-ruby action supports this feature and will handle 'bundle install' automatically when caching is enabled.

Suggested change
- name: Install dependencies
run: |
gem install bundler
bundle install
bundler-cache: true
- name: Install dependencies
run: echo "Dependencies are installed via ruby/setup-ruby with bundler-cache"

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants